@stephenwithav this pull request has failed checks 🛠

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.01%. Comparing base (5a80a04) to head (5a1e0d9).
Report is 121 commits behind head on master.

@@             Coverage Diff             @@
##           master    #2490       +/-   ##
===========================================
+ Coverage   61.56%   75.01%   +13.45%     
===========================================
  Files          53       62        +9     
  Lines        9002    10008     +1006     
===========================================
+ Hits         5542     7508     +1966     
+ Misses       3020     1937     -1083     
- Partials      440      563      +123     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

The OS/X check failed.

Only a GitHub Actions infra degregation, rerunning now.

Artifact upload failed.

Another approval till merge.

This PR works, but act -q is closer to what I wanted.

@ChristopherHX @stephenwithav this appears ready to go, will this be merged soon? Thanks!

@mcascone I respect the merge requirements of this project, unless there is a regression with major impact or no reviews for n-months. Where n is an natural number > 1, the latter never happend yet.

Merge requirement for this pr are two maintainer approvals for outside contributions.

@mcascone I respect the merge requirements of this project, unless there is a regression with major impact or no reviews for n-months. Where n is an natural number > 1, the latter never happend yet.

Merge requirement for this pr are two maintainer approvals for outside contributions.

I totally understand this. So in a week or three, if there are no more reviews, it'll be mergeable?